Skip to content

perf: enable rspack's nativeWatcher#518

Draft
9aoy wants to merge 12 commits into
mainfrom
nativeWatcher
Draft

perf: enable rspack's nativeWatcher#518
9aoy wants to merge 12 commits into
mainfrom
nativeWatcher

Conversation

@9aoy
Copy link
Copy Markdown
Collaborator

@9aoy 9aoy commented Aug 28, 2025

Summary

enable rspack's nativeWatcher.

https://rspack.rs/config/experiments#experimentsnativewatcher

Related Links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@netlify
Copy link
Copy Markdown

netlify Bot commented Aug 28, 2025

Deploy Preview for rstest-dev ready!

Name Link
🔨 Latest commit 72fee8a
🔍 Latest deploy log https://app.netlify.com/projects/rstest-dev/deploys/693a776efab34d00089c9ca3
😎 Deploy Preview https://deploy-preview-518--rstest-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@9aoy 9aoy marked this pull request as draft August 28, 2025 03:45
@chenjiahan chenjiahan changed the title chore: enable rspack's nativeWatcher perf: enable rspack's nativeWatcher Oct 29, 2025
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Apr 16, 2026

Deploying rstest with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6569ac4
Status: ✅  Deploy successful!
Preview URL: https://a6dbdd2e.rstest.pages.dev
Branch Preview URL: https://nativewatcher.rstest.pages.dev

View logs

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 16, 2026

Rsdoctor Bundle Diff Analysis

Found 12 projects in monorepo, 1 project with changes.

📊 Quick Summary
Project Total Size Change
adapter-rsbuild 3.6 KB 0
adapter-rslib 24.6 KB 0
adapter-rspack 7.7 KB 0
browser 2.0 MB 0
browser-react 3.7 KB 0
browser-ui 803.5 KB 0
coverage-istanbul 9.6 KB 0
core/browser 970.0 KB 0
core/loaders 869.0 B 0
core/main 1.6 MB +108.0 B (0.0%)
vscode/extension 58.9 MB 0
vscode/worker 14.4 KB 0
📋 Detailed Reports (Click to expand)

📁 core/main

Path: packages/core/.rsdoctor/main/rsdoctor-data.json

📌 Baseline Commit: 34bbcd8169 | PR: #1247

Metric Current Baseline Change
📊 Total Size 1.6 MB 1.6 MB +108.0 B (0.0%)
📄 JavaScript 1.5 MB 1.5 MB +108.0 B (0.0%)
🎨 CSS 0 B 0 B 0
🌐 HTML 0 B 0 B 0
📁 Other Assets 79.2 KB 79.2 KB 0

📦 Download Diff Report: core/main Bundle Diff

Generated by Rsdoctor GitHub Action

@codspeed-hq
Copy link
Copy Markdown
Contributor

codspeed-hq Bot commented Apr 16, 2026

Merging this PR will not alter performance

✅ 1 untouched benchmark
⏩ 3 skipped benchmarks1


Comparing nativeWatcher (55a7bf6) with main (3fea26b)

Open in CodSpeed

Footnotes

  1. 3 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@stormslowly stormslowly self-assigned this May 11, 2026
stormslowly added a commit to stormslowly/rstest that referenced this pull request May 15, 2026
Mirrors the one-line change from web-infra-dev#518 so the e2e matrix on this branch
exercises the nativeWatcher code path while validating the fresh fixture
directory strategy. Used to confirm the prepareFixtures change in the
prior commit eliminates the flaky "restarting Rstest as
rstest.config.mts changed" failure under nativeWatcher.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants